Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[improve][broker] Fix ServiceUnitStateCompactionStrategy to cover fas… #46

Closed
wants to merge 4 commits into from

Conversation

heesung-sn
Copy link
Owner

@heesung-sn heesung-sn commented Apr 15, 2023

Master Issue: apache#16691

Motivation

Raising a PR to implement: apache#16691

Modifications

This PR

  • Accept the msg if its version is bigger than the current +1

Verifying this change

  • Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

  • *Updated unit tests.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

We will have separate PRs to update the Doc later.

Matching PR in forked repository

PR in forked repository: #45

@heesung-sn heesung-sn closed this Apr 18, 2023
@heesung-sn heesung-sn deleted the pip-192-fix-tv-consistency branch April 2, 2024 17:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant